[Aikido] Fix 40 security issues in fast-uri, hono - #10
Closed
aikido-autofix[bot] wants to merge 1 commit into
Closed
[Aikido] Fix 40 security issues in fast-uri, hono#10aikido-autofix[bot] wants to merge 1 commit into
aikido-autofix[bot] wants to merge 1 commit into
Conversation
Author
|
Closed by Aikido: a new AutoFix has been created → #11 |
aikido-autofix
Bot
deleted the
fix/aikido-security-update-packages-79615312-7w4t
branch
August 5, 2026 00:17
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Upgrade fast-uri and hono to fix critical SSRF/host-bypass and path-traversal vulnerabilities caused by improper URI normalization and authority parsing.
✅ No breaking changes for: fast-uri
✅ 40 CVEs resolved by this upgrade
This PR will resolve the following CVEs:
fast-uritreats as paths but Node's URL parser treats as authority, enabling redirection to unintended destinations.Content-Lengthwas missing or bodies were chunked, due to asynchronous limit enforcement. This could enable DoS attacks or unintended data processing despite configured size restrictions.setCookie(),serialize(), orserializeSigned(), allowing invalid characters that can cause malformed Set-Cookie headers and runtime errors when processing untrusted cookie names.parseBody({ dot: true })where specially crafted form field names like__proto__.xcreate objects with__proto__properties, potentially enabling prototype pollution if merged unsafely into other objects.jsx()andcreateElement(), allowing untrusted tag input to inject markup and reshape generated HTML, potentially enabling XSS attacks.🤖 Remediation details
Fix high-severity vulnerabilities in
fast-uriandhonovia lockfile-only resolution refreshShort summary
This PR remediates multiple high- and medium-severity vulnerabilities in two packages:
fast-uri(URI parsing) andhono(web framework). Both packages were already reachable within the workspace —honoas a direct dependency ofapps/worker, andfast-urias a transitive dependency pulled in throughconf→ajv— and both were resolved to vulnerable versions inpnpm-lock.yaml. No manifest (package.json) edits were required; all fixes were applied via lockfile-only resolution updates.fast-uri
fast-uriis a transitive dependency resolved throughapps/cli's direct dependencyconf@13.1.0, which depends onajv@8.18.0, which in turn declaresfast-uri: "^3.0.1". The lockfile had stale-pinnedfast-uriat3.1.0, a version within multiple vulnerable ranges. Becauseajv@8.18.0's declared range^3.0.1already permits3.1.5, no manifest change was needed — runningpnpm update fast-uri --recursive --lockfile-onlywas sufficient to re-resolve the transitive to3.1.5, the minimum version that clears all identified advisories for this package.hono
honois a direct dependency ofapps/worker, declared as^4.7.0in that workspace member'spackage.json. The lockfile had resolved it to4.12.5, which falls within the vulnerable ranges for all identified advisories. The declared caret range already permits4.12.25and above, so no manifest edit was required; a lockfile-only update viapnpm update --latest hono --recursive --lockfile-onlyadvanced the resolved version to4.12.33, clearing all identified advisories for this package.Version changes
hono4.12.54.12.33^4.7.0rangefast-uri3.1.03.1.5ajv@8.18.0— lockfile refresh within existing^3.0.1rangeNote
Medium Risk
Dependency-only changes on the worker’s core HTTP framework (
hono); behavior should be patch-level but warrants smoke-testing API, auth middleware, and SSR views after deploy.Overview
This PR addresses security advisories by upgrading
honoon the Cloudflare worker and refreshing transitive resolution forfast-uriin the lockfile.In
apps/worker, the directhonodependency moves from^4.7.0(lockfile had4.12.5) to^4.12.33, which is what actually runs the worker’s routing, middleware factory, and JSX/HTML rendering paths.The
pnpm-lock.yamlupdate also pinsfast-urifrom3.1.0to3.1.5(pulled in viaajv/ CLI tooling), and includes incidental lockfile churn (e.g.libcmetadata on native optional packages,viteno longer listing an optionaltsxpeer, and CLI entries forhighlight.js/marked-highlightif those were part of the same refresh).Reviewed by Cursor Bugbot for commit 51e7480. Bugbot is set up for automated code reviews on this repo. Configure here.